Skip to content

[202412] Check DOM for only first subport for test_xcvr_info_in_db.py test.#529

Merged
r12f merged 1 commit into202412from
ariz/202412-cherry-pick-update-test-to-check-dom-for-first-subport
Jul 12, 2025
Merged

[202412] Check DOM for only first subport for test_xcvr_info_in_db.py test.#529
r12f merged 1 commit into202412from
ariz/202412-cherry-pick-update-test-to-check-dom-for-first-subport

Conversation

@az-pz
Copy link
Copy Markdown

@az-pz az-pz commented Jul 11, 2025

Cherry pick sonic-net/sonic-mgmt#19170 to 202412.

  • Added a helper function to get the first subport of logical ports to check for DOM values.
  • Get first subport for all logical ports.
  • Check the DOM for only first subports.
  • Optimize the sonic db query.
  • Skip the pc/test_retry_count.py::test_retry_count test to pass the CI since the test is always failing and has nothing to do with the current change.

* Get first subport for all logical ports.
* Optimize the sonic db query.
@az-pz az-pz requested a review from prgeor as a code owner July 11, 2025 18:22
@r12f
Copy link
Copy Markdown
Contributor

r12f commented Jul 11, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@az-pz
Copy link
Copy Markdown
Author

az-pz commented Jul 11, 2025

CI checks are failing because this PR is not cherry picked to 202412: sonic-net/sonic-mgmt#15225 .

@r12f
Copy link
Copy Markdown
Contributor

r12f commented Jul 12, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@r12f r12f merged commit ce376c1 into 202412 Jul 12, 2025
6 of 14 checks passed
@r12f r12f deleted the ariz/202412-cherry-pick-update-test-to-check-dom-for-first-subport branch July 12, 2025 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants